Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Winter Break 2021-2022 Master Push #69

Merged
merged 25 commits into from
Jan 4, 2022
Merged

Winter Break 2021-2022 Master Push #69

merged 25 commits into from
Jan 4, 2022

Conversation

charlestian23
Copy link
Collaborator

@charlestian23 charlestian23 commented Jan 4, 2022

Summary of changes in this push:

charlestian23 and others added 25 commits December 25, 2021 20:46
Moved the invalid use error message to the general Rule class
Now displays an error message when there is an invalid use of a case rule
Fixed checkRuleRaw() so that ShortTruthTable case rules can no longer be used to justify all moves; still need to bug test
The number of cases in a case rule must be greater than 1, as otherwise, there wouldn't be multiple cases. Now, the code only enters the if statement if the number of cases is more than 1.
Adjusted the invalid use message for case rules
Implemented Nurikabe error messages and adjusted global error message wordings
Implemented Light Up error messages and adjusted global error message wordings
Implemented standardized error messages for Fillapix
Fixed one of the error messages for Short Truth Table
Standardized error messages for Skyscrapers
Standardized Sudoku error messages
Standardized Tree Tent error messages
Missed some Nurikabe rules in the first go-around
Added MacOS shortcut support
…-restructuring

Standardized Error Messages
@Chase-Grajeda Chase-Grajeda merged commit ba2ae10 into master Jan 4, 2022
@charlestian23 charlestian23 added the semester push For the final master push of the working semester label Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semester push For the final master push of the working semester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants